Skip to content

[Geant4_julia] Use latest Geant4_jll 11.3 #11098

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom
Open
wants to merge 3 commits into
base: master
Choose a base branch
from
Open

[Geant4_julia] Use latest Geant4_jll 11.3 #11098

wants to merge 3 commits into from

Conversation

peremato
Copy link
Contributor

No description provided.

@giordano giordano enabled auto-merge (squash) May 12, 2025 12:33
@peremato
Copy link
Contributor Author

@giordano perhaps you can help me here. I'm facing two issues:

  • For apple-darwin, when the package Geant4_jll was built, it found ZLIB version 1.3.1 in the "system". However, now when building this package, it finds version 1.2.11 in the "system", which causes a conflict. A test build for x86_64-apple-darwin works, but the one for x86_64-apple-darwin-julia_version+1.11 fails. Any idea why this is happening?
  • It also tries to build for julia_version+1.13.0, but fails—presumably because CxxWrap is not available for that version.

Any suggestions on how to resolve these issues? Thanks!

giordano
giordano reviewed May 24, 2025
View reviewed changes
G/Geant4_julia/build_tarballs.jl
@@ -56,14 +57,12 @@ products = [
# Dependencies that must be installed before this package can be built
dependencies = [
BuildDependency("libjulia_jll"),
BuildDependency("Xorg_xorgproto_jll"),
Dependency("libcxxwrap_julia_jll"; compat="0.13.2"),
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

You may want to try version 0.14

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

yes, sure I can try to use version 0.14. However if I do this, then I would have to upgrade all "binding" packages using CxxWrap. If I want to use, for example, Geant4 together with PYTHIA8, which I do in one of the examples, I would need to update PYTHIA8.

giordano
giordano reviewed May 24, 2025
View reviewed changes
G/Geant4_julia/build_tarballs.jl
@@ -24,7 +24,8 @@ cmake ${WORKSPACE}/srcdir/Geant4_cxxwrap -B build \
-DCMAKE_INSTALL_PREFIX=${prefix} \
-DCMAKE_TOOLCHAIN_FILE=${CMAKE_TARGET_TOOLCHAIN} \
-DCMAKE_BUILD_TYPE=Release \
-DCMAKE_CXX_STANDARD=17
-DCMAKE_CXX_STANDARD=17 \
-DZLIB_ROOT="${prefix}"
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

It doesn't look like this option is being used or effective. One option is to delete the tbd file in the sysroot, but I think this is a cmake issue.

@barche
Copy link
Contributor

barche commented May 24, 2025

Versions 0.13.4 and 0.14.2 of libcxxwrap_julia_jll support Julia 1.13. I recommend using the 0.14 series since the new CxxWrap (0.17) using it is released, unless you need to interoperate with other CxxWrap-based packages that haven't upgraded yet.

@peremato
Copy link
Contributor Author

peremato commented Jun 3, 2025

Versions 0.13.4 and 0.14.2 of libcxxwrap_julia_jll support Julia 1.13. I recommend using the 0.14 series since the new CxxWrap (0.17) using it is released, unless you need to interoperate with other CxxWrap-based packages that haven't upgraded yet.

yes, this what I mentioned in the previous comment. I need to upgrade to new CxxWrap all packages at the same time.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@giordano giordano giordano left review comments

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
3 participants
@peremato @barche @giordano